Skip to content

Conversation

@En-En-Code
Copy link
Contributor

Thanks for getting back quickly to my response. Honestly was not expecting the quick turn-around.

Since it being merged makes issues like these within RustSec's purview, I would like to add io-safety as a free-form keyword, hopefully to set a precedent that I/O safety violations are problems which should be reported.

@djc
Copy link
Contributor

djc commented Sep 1, 2025

Seems okay, but the branch has conflicts. Want to rebase?

@En-En-Code En-En-Code force-pushed the xcb-io-safety-keyword branch from 393eb1c to cbe2863 Compare September 29, 2025 14:35
@En-En-Code
Copy link
Contributor Author

Sorry for letting this slide so far down my priority list for how simple it was. (Well, maybe it wasn't that simple since my first rebase made a mess without fixing the conflict, but nothing a force branch reset and force push can't "fix". Hey, it got there in the end.)

@djc djc merged commit f71b77f into rustsec:main Sep 30, 2025
1 check passed
@djc
Copy link
Contributor

djc commented Sep 30, 2025

Sorry for letting this slide so far down my priority list for how simple it was. (Well, maybe it wasn't that simple since my first rebase made a mess without fixing the conflict, but nothing a force branch reset and force push can't "fix". Hey, it got there in the end.)

Better late than never!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants